Skip to content

Support for multipart requests, new ways of routing the request to MP graph #3250

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 29 commits into
base: main
Choose a base branch
from

Conversation

dkalinowski
Copy link
Collaborator

@dkalinowski dkalinowski commented Apr 18, 2025

🛠 Summary

CVS-163721

If request content-type is application-json - route based on model JSON field
If request content-type s multipart - route based on mode multipart field
In other cases try to deduce graph name from URI: /v3/GRAPH-NAME

  • multipart field/file access for calculators
  • unit tests

🧪 Checklist

  • Unit tests added.
  • Change follows security best practices.

@dkalinowski dkalinowski added the WIP Do not merge until resolved label Apr 18, 2025
@dkalinowski dkalinowski requested review from atobiszei and mzegla April 22, 2025 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Do not merge until resolved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant